-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(deps): update deps/open62541 digest to 258d6ad #1080
Conversation
a89e368
to
09ccca7
Compare
09ccca7
to
3a7256d
Compare
c365a06
to
0fc218d
Compare
Looks like, there is a new method for loading companion specifications: I don't known why the original method is broken. |
04b3fb8
to
3902c32
Compare
@NoelGraf do you have an idea, why the old nodeset generation is now failing? |
30de62f
to
5c481f4
Compare
Each CS without types causes this is the error: #define UA_TYPES_PADIM_COUNT 0
#define UA_TYPES_PADIM NULL /* Change namespaceIndex from current namespace */
for(int i = 0; i < UA_TYPES_PADIM_COUNT; i++) {
UA_TYPES_PADIM[i].typeId.namespaceIndex = ns[2];
UA_TYPES_PADIM[i].binaryEncodingId.namespaceIndex = ns[2];
} |
5c481f4
to
da1387e
Compare
This is a bug in the current open62541 release: open62541/open62541#5814 |
da1387e
to
dcd911b
Compare
4f90e9b
to
6aaebc8
Compare
The original Error seems to be gone, but now GMS has a problem:
|
True, I'm currently investigating if related to NodeSet or open62541 Generator. Because the GMS nodeset currently processed is the RC NodeSet. Thanks for looking. |
dbfea31
to
7880088
Compare
If adjusted to the release versions of Machinery/Result and GMS the following error remains:
@NoelGraf I suspect the the new nodeset generator cannot deal with the |
I tried to build the gms nodeset and it worked. |
Thank you @NoelGraf! Will investigate further. |
960bf8f
to
c9af2bb
Compare
04e54f4
to
a272d40
Compare
a272d40
to
39c5182
Compare
06b87a7
to
c396dc0
Compare
The Machinery Result bsd-File (and xsd) does not specify a type for
The original enum has been removed, the author of this may need to change this to the updated values. Lines 116 to 117 in c396dc0
|
c396dc0
to
cc5caaf
Compare
23f4bb0
to
28c1d65
Compare
* Fix Typo in Identifier
28c1d65
to
2ab5704
Compare
This PR contains the following updates:
7d602cb
->258d6ad
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Mend Renovate. View repository job log here.